Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

document how branchIndex works #2167

Merged
merged 2 commits into from
Jun 21, 2024
Merged

Conversation

despairblue
Copy link
Contributor

@despairblue despairblue commented Jun 14, 2024

This documents the changes made here: n8n-io/n8n#9760

Changed page: https://docs.n8n.io/code/builtin/output-other-nodes/

Copy link

netlify bot commented Jun 14, 2024

Deploy Preview for n8n-docs ready!

Name Link
🔨 Latest commit 5def197
🔍 Latest deploy log https://app.netlify.com/sites/n8n-docs/deploys/667191944946950008827b32
😎 Deploy Preview https://deploy-preview-2167--n8n-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

github-actions bot commented Jun 14, 2024

Overall readability score: 41.92 (🟢 +0.01)

File Readability
output-other-nodes.md 63.5 (🟢 +7.01)
View detailed metrics

🟢 - Shows an increase in readability
🔴 - Shows a decrease in readability

File Readability FRE GF ARI CLI DCRS
output-other-nodes.md 63.5 51.07 9.4 14.5 10.27 6.21
  🟢 +7.01 🔴 -0.31 🟢 +0.29 🟢 +2.1 🟢 +1.33 🟢 +0.51

Averages:

  Readability FRE GF ARI CLI DCRS
Average 41.92 34.81 12.02 15.15 14.26 8.64
  🟢 +0.01 🟢 +0 🟢 +0 🟢 +0 🟢 +0 🟢 +0
View metric targets
Metric Range Ideal score
Flesch Reading Ease 100 (very easy read) to 0 (extremely difficult read) 60
Gunning Fog 6 (very easy read) to 17 (extremely difficult read) 8 or less
Auto. Read. Index 6 (very easy read) to 14 (extremely difficult read) 8 or less
Coleman Liau Index 6 (very easy read) to 17 (extremely difficult read) 8 or less
Dale-Chall Readability 4.9 (very easy read) to 9.9 (extremely difficult read) 6.9 or less

@despairblue despairblue force-pushed the pay-1400-bug-last-uses-wrong-branch branch from c2865e0 to 5def197 Compare June 18, 2024 13:54
@despairblue despairblue added the in-next-release Follow-up to pending-dev. Use once the linked dev PR is merged. label Jun 19, 2024
@despairblue
Copy link
Contributor Author

@StarfallProjects the change has been merged and should be in 1.47. Let me know if the explanation is confusing. Happy to amend it.

I added the label in-next-release. I hope that's correct.

@Joffcom Joffcom marked this pull request as ready for review June 21, 2024 07:12
@Joffcom Joffcom merged commit 4398dbb into main Jun 21, 2024
7 checks passed
@Joffcom Joffcom deleted the pay-1400-bug-last-uses-wrong-branch branch June 21, 2024 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-next-release Follow-up to pending-dev. Use once the linked dev PR is merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants